Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

0183-WebUI-ImprovedFirmwareupdateDialog #2532

Merged
merged 10 commits into from
Jan 27, 2024

Conversation

Baxxy13
Copy link
Contributor

@Baxxy13 Baxxy13 commented Nov 27, 2023

Description

Modified "new DeviceFirmware PopUp".

Related Issue

#2396 (comment)

Types of changes

  • Docs change / refactoring / dependency upgrade
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Alternate Designs

Possible Drawbacks

Nothing known / found so far.

Verification Process

I'm using this patch since some months on different RaspberryMatic-Systems. All is working as expected.

Release Notes

Reworked colorized design which groups same devices and count them.
Small help and shortcut for updating implemented.

Contributing checklist

  • My code follows the code style of this project.
  • I have read the CONTRIBUTING and LICENSE document.
  • I fully agree to distribute my changes under Apache 2.0 license.

@Baxxy13
Copy link
Contributor Author

Baxxy13 commented Nov 27, 2023

Aktuell sieht das so aus:
(ohne die gelbe Markierung)
FW-Update_SWDO-I

@jens-maus
Copy link
Owner

Wie du siehst lässt sich aktuell der neue WebUI Patch nicht anwenden (weil in einem ähnlichen Bereich schon in vorherigen WebUI patches Änderungen vorgenommen wurden). Siehe

https://github.com/jens-maus/RaspberryMatic/actions/runs/7009367224/job/19067636364?pr=2532#step:13:7108

Müsstest du also noch einmal einarbeiten.

@Baxxy13
Copy link
Contributor Author

Baxxy13 commented Nov 27, 2023

Ok.
Das ist für mich Neuland. Wie gehe ich das jetzt an?

@IT-VBFK
Copy link
Contributor

IT-VBFK commented Nov 28, 2023

Wenn ich bisherige Hilfen (von Jens und Jerome) richtig verstehe, dann musst du als webui.js.orig die webui.js aus dem letzten Patch (zB Patch 173) nehmen, der in diesem Bereich was verändert (und dann darauf aufbauend deine Änderungen machen)

@jens-maus
Copy link
Owner

jens-maus commented Nov 28, 2023

Wenn ich bisherige Hilfen (von Jens und Jerome) richtig verstehe, dann musst du als webui.js.orig die webui.js aus dem letzten Patch (zB Patch 173) nehmen, der in diesem Bereich was verändert (und dann darauf aufbauend deine Änderungen machen)

Das wäre ein Ansatz, ja. Und vermutlich auch der einfachste für @Baxxy13.
D.h. einfach mal schrittweise durch alle WebUI patch nummern kleiner als die neue durchgehen und in dem Patch wo die webui.js das letzte mal abgeändert wurde diese dann im neuen Patch als webui.js.orig datei nutzen und darauf basierend dann die Anpassungen erneut umsetzen.

Alternativ kann man auch einfach schauen welcher "Hunk" des patch files sich nicht anwenden lässt bzw. welcher zu einem fehler führt und dann die *.rej Datei in dem jeweiligen Verzeichnis anschauen und die notwendigen Änderungen dann in der *.orig Datei einpflegen damit das patch file sich wieder anwenden lässt.

Hilfreich bei der ganze Sache ist auch immer make check im top-level verzeichnis des repositories auszuführen. Denn dieser Befehl macht genau das was die CI/CD Engine hier in GitHub auch macht, es prüft ob alle patches sich korrekt wenden lassen oder ob irgendwo in der Patch chain ein fehler existiert.

@Baxxy13
Copy link
Contributor Author

Baxxy13 commented Nov 28, 2023

Ah ja, sowas hatte ich mir gestern schon gedacht und das mal mit der webui.js aus 0127 probiert.
Das klappte aber ebenso nicht, ich muss also eine "spätere Version" nehmen.

Wenn ich das richtig gesehen habe ist 0181 der aktuell "letzte" Patch der die webui.js anfasst.
Von dem würde ich jetzt die webui.js als webui.js.orig nehmen.

Das mit den .rej Files klingt mir erstmal zu kompliziert, ich gehe den einfachen Weg.

@jens-maus
Copy link
Owner

Probier es. Zu beachten gilt nur, das du sicherstellst nicht einfach copy&paste zu machen, sondern gerade wenn in der exakt selben Zeile in vorherigen WebUI patches änderungen vorgenommen wurden du diese änderungen nicht mit dem neuen Patch quasi rückgängig machst oder Funktionen damit veränderst...

@Baxxy13
Copy link
Contributor Author

Baxxy13 commented Nov 28, 2023

Na hoffentlich steht in den nächsten Release-Notes nicht "Baxxy hat die RaspberyMatic kaputt gemacht". 😉

Wie finde ich denn den Patch der im betreffenden Bereich Änderungen gemacht hat?

@jens-maus
Copy link
Owner

Wie finde ich denn den Patch der im betreffenden Bereich Änderungen gemacht hat?

Du hast ihn ja anscheinend gefunden, denn der CI Build Check lief ja jetzt nach deinen letzten Anpassungen sauber durch.. sehr gut :)

@jens-maus jens-maus added 💡 enhancement-ideas New feature or change request 🏷️ WebUI This refs the WebUI component labels Nov 28, 2023
@jens-maus jens-maus added this to the next release milestone Jan 27, 2024
@jens-maus jens-maus merged commit 8c9e208 into jens-maus:master Jan 27, 2024
4 checks passed
@Baxxy13 Baxxy13 deleted the FW_Update_Dialog branch January 29, 2024 08:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💡 enhancement-ideas New feature or change request 🏷️ WebUI This refs the WebUI component
Projects
Development

Successfully merging this pull request may close these issues.

3 participants